Don't use g_slist_next in gtktextlayout.c

We generally use ->next directly.

Don't use g_slist_next in gtktextdisplay.c

We generally use ->next directly.

Don't use g_slist_next in gtktextchild.c

We generally use ->next directly.

Don't use g_slist_next in gtktextbufferrichtext.c

We generally use ->next directly.

Don't use g_slist_next in gtktextbuffer.c

We generally use ->next directly.

Don't use g_slist_next in gtkstock.c

We generally use ->next directly.

Don't use g_slist_next in gtkiconfactory.c

We generally use ->next directly.

Don't use g_slist_next in the x11 backend

We generally use ->next directly.

Don't use g_slist_next in the windows backend

We generally use ->next directly.

Don't use g_slist_next in gdk

We generally just use ->next directly.

Don't use g_list_next in gtk3-demo

We generally just use ->next directly.

Don't use g_list_next in gtkselection.c

We generally access ->next directly.

Don't use g_list_next in gtkmain.c

We generally access ->next directly.

Don't use g_list_next in gtkdialog.c

We generall access ->next directly.

Don't use g_list_next in gtkcontainer.c

We generally access ->next directory.

Don't use g_slist_next in gtktextbtree.c

We generally access ->next directly.

builder: Don't use g_slist_next

We just use direct access to list->next all over the place.

builder: Cosmetic changes

Use an iter instead of g_hash_table_forall.

Cosmetic changes

Line up struct-filling code with the order in the declaration.

Add some more builder parser tests

These tests contain non-canonical and non-existing property names.

builder: Avoid some unnecessary overhead

Only get the class once per object, not once per property.
And don't canonicalize the property name, g_object_class_find_property
does that already.

Updated Basque language

Updated Basque language

widget: fix typo in warning message

wayland: Map windows with tooltip hint as subsurfaces

Tooltips tend to be placed on top of a parent surface with a given
relative coordinate, and without any input focus. So lets map them as
subsurfaces.

https://bugzilla.gnome.org/show_bug.cgi?id=756496

wayland: Clean up code determining how to map a window

Restructure the mapping procedure so that its known up front what the
expected way mapping is to be done (subsurface, popup or stand alone),
and warn if it fails to actually map in such a way (for example a popup
without a parent or device grab, a tooltip without a parent).

https://bugzilla.gnome.org/show_bug.cgi?id=756496

Minor cleanup

Replace a unref-and-unset combination with g_clear_object.

Clean up handling of _gdk_win32_grab_cursor

This is a variable holding a ref to an object, so it is
a great case to use g_set_object and g_clear_object.

 # Please enter the commit message for your changes. Lines starting

W32: Add missing reference taking in gdk_device_virtual_grab()

https://bugzilla.gnome.org/show_bug.cgi?id=756160

Updated Norwegian bokmål translation.

Fixes from Einar Ulvsløkk.

GtkPaned: Fix gtk_paned_set_child_visible calls

Use CHILD1/CHILD2 instead of 0 and 1, always use the same order and
don't check for child NULL-ness, because it will be done in
gtk_paned_set_child_visible anyways.

GtkRevealer: Remove useless _start_animation call

The call just passed priv->target_pos, but _start_animation directly
returns if the given target is priv->target_pos (see line 627).

mir: send touch events

win32: fix warnings about signed/unsigned mismatch

win32: use the same type as the prototype

inspector: remove gtk_text_buffer_get_iter_at_line_index() workaround

The function is now safe to use.

https://bugzilla.gnome.org/show_bug.cgi?id=735341

textbuffer: nicer get_iter functions

Avoid crashes when passing an invalid location to a
gtk_text_buffer_get_iter_at_*() function.

A boolean is returned to know if @iter has been set to the exact
location.

Unit tests are added.

https://bugzilla.gnome.org/show_bug.cgi?id=735341

Updated Slovak translation

Updated Slovak translation

Split off a private header for GtkTextBuffer

This avoids polluting the installed header with private symbols.

Improve text a11y test

Test appending text as well as replacing the entire text.
Insert non-ASCII text to verify offsets are correct.

file chooser: Avoid search interruption

When the search entry is shown, the 'special' nature of
., ~ and / should not trigger the location entry, because
that interrupts the search and is likely not what the
user intended.

https://bugzilla.gnome.org/show_bug.cgi?id=756505

entry completion: Only grab the device if we have one

When the entry completion is popped up from a timeout, we may
not have a device. In that case, don't call gdk_device_grab,
do avoid criticals.

scrolled window: Protect against nameless devices

It seems that gdk_device_get_name() can return NULL.
We should not crash if that happens.

https://bugzilla.gnome.org/show_bug.cgi?id=756625

gtkplacesview: align spinner with header label

Use the box margin top instead of the label margin top,
so the spinner remains aligned with the header label.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: plug leak

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: remove hover color from rows

Since other views are not using hover neither

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: add a clear button to address entry

So it allows a quick way to clear the entry.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: rotate server list icon on toggled

Disclosure triangles are usually used pointing down, however
in this case the popover spawns in the upper direction, which
makes it odd looking.
Instead of pointing always down or up, point down when not toggled and
animate a rotation when toggled.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: tweak ui to allow more server rows

Following design guidance, reduce row height and increase
popover height so the user is allowed to see more than 3
rows.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: improve heuristics for external drives

Following the sidebar on commit b0989b190df, improve the way
we check when a drive is external or not.

https://bugzilla.gnome.org/show_bug.cgi?id=756589

gtkplacesview: remove dead code

This is checked on add_volume inside the loop.

https://bugzilla.gnome.org/show_bug.cgi?id=756589

gtkplacesview: remove dead code

This is checked on is_removable_volume

https://bugzilla.gnome.org/show_bug.cgi?id=756589

gtkplacessidebar: improve heuristics for external drives

Since the change to use GtkPlacesView we don't want to show
internal storage on the sidebar.

In our case we were checking for drive_can_eject and
drive_is_media_removable.

However for some external hard drives it's reported that they
are not ejectable nor the have removable media. So the only
attribute that they have different from internal drives is that
they can be stopped.
So check for if the drive can be stopped to decide if it is
external or internal.

On the way realized we don't need to check for the mounts associated
with the volume to know if the volume can be ejected or not. So remove
that code.

https://bugzilla.gnome.org/show_bug.cgi?id=756589

mir: add mircookie as a dependency

texthandle: Ensure handles are invalidated on mode changes

Otherwise the "cursor" handle stays with "cursor" appearance instead of
"selection end" when a text selection is started.

texthandle: small refactor

These long enums are used too often, shorten things a bit with temp vars.

texthandle: Fix Y positioning of text handles

It is assumed that border.top is the same than pointing_to.height (which
equals the strong cursor position), which is not since some time ago.

The border calculation has been move on top too, it is now used in the
Y position one, and doesn't depend on anything we calculate later.

texthandle: Fix handle dragging on wayland

Text handles are subsurfaces on wayland, so sort of their own toplevel.
This made gtk_widget_translate_coordinates() to bail out there, resulting
in text handles being mispositioned and jumpy. To fix this, translate to
toplevel GtkWindow coordinates manually, and translate coordinates from
there.

Along the way, the coordinates reported in ::handle-dragged have been
fixed so there is no small jumps in either axis (most noticeable in the
X axis when you started dragging, and in the Y axis when moving between
lines of different heights.

gtkplacessidebar: Show drives with removable media

Some drives have removable media that is not ejectable (e.g. a laptop's
SD card reader). Show volumes on these drives in the sidebar.

https://bugzilla.gnome.org/show_bug.cgi?id=755654

Updated Basque language

mir: support more pixel formats

GtkWindow: Enlarge the type hint private field

Make it what it is - the enum - so that that it is sure that the hint
will fit in the field. Without this, any hint that doesn't fit in 3
bits will be truncated to the 3 least significant bits, causing
unexpected behaviour.

https://bugzilla.gnome.org/show_bug.cgi?id=756496

Updated Czech translation

GdkAppLaunchContext: Use the display name for X11

Using a NULL GAppInfo with g_app_launch_context_get_display() will
generate a critical warning in gio.

Use the display name instead as we don't have any valid GAppInfo to pass
to g_app_launch_context_get_display().

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=756439

textview: Ensure magnifier popover points within widget area

Otherwise the popopver will be automatically unmapped in
_gtk_popover_update_child_visible() when the X axis (coming more
or less directly from events) goes outside the textview.

popover: Invalidate borders on tail position changes

There's nothing doing this explicitly, so tail position changes may
leave trails of older positions.

https://bugzilla.gnome.org/show_bug.cgi?id=756449

wayland: Set a more believable crossing detail on pointer enter/leave

GDK_NOTIFY_ANCESTOR would happen when the pointer crosses across a direct
parent/child. However nonlinear events are more likely, specially when
the pointer moves across toplevels (either different apps, or menus being
popped up over the pointer position).

This makes popping up comboboxes and other menus that fall over the pointer
position possible. With the previous detail the GtkMenu code misinterpreted
the crossing event, making it think the button release coming right after
should dismiss the popup, which made menus just flash on the screen unless
you kept the button pressed.

gtk-query-settings: Add license header

MSVC builds: Fix "install"

Use $(GlibEtcInstallRoot) when invoking glib-compile-schemas, as CopyDir
is not GlibInstallRoot for GTK+ (due to quoting issues), meaning that the
glib-compile-schemas tool may not be found in certain cases.

Issue pointed out by Ignacio Casal Quinteiro.

Add gtk-query-settings

A small utility binary to query all properties of GtkSettings.

https://bugzilla.gnome.org/show_bug.cgi?id=756174

stylecontext: Make sure style is valid when looking up style properties

Otherwise, the validation may happen vey automatically some time during
the style property and that will most likely cause a crash.

https://bugzilla.gnome.org/show_bug.cgi?id=756338

Adwaita: osd spinbutton fixes

Adwaita: stop consider buttons on osd 'linked'

we used to consider every button inside osd containers linked,
this is not true anymore, now those buttons behave normally.
This will clearly cause breakage in applications.

gtkprintoperation: job names must not exceed 255 chars

According to http://datatracker.ietf.org/doc/rfc2911/, The 'name'
attribute syntax is essentially the same as 'text', including the
REQUIRED support of UTF-8 except that the sequence of characters
is limited so that its encoded form MUST NOT exceed 255 (MAX) octets.

CUPS will not print jobs with names exceeding 255 characters.

https://bugzilla.gnome.org/show_bug.cgi?id=755988

im context simple: Add documentation

Document that GtkIMContextSimple is loading additional compose
tables. Also document Ctrl-Shift-u.

GtkIMContextSimple: Load locale compose tables dynamically.

Load /usr/share/X11/locale/$LOCALE/Compose and $HOME/.XCompose
dynamically.

https://bugzilla.gnome.org/show_bug.cgi?id=721120

Add gtk_compose_table_new_with_file() to create GtkComposeTable

https://bugzilla.gnome.org/show_bug.cgi?id=721120

GtkIMContextSimple: Change guint to guint16 in gtkimcontextsimple.c

GtkComposeTable and GtkComposeTableCompact use guint16 for the data.

https://bugzilla.gnome.org/show_bug.cgi?id=721120

GtkIMContextSimple: export _gtk_check_compact_table() and GtkComposeTable

Some functions need to be exported if X11 compose files are loaded.

https://bugzilla.gnome.org/show_bug.cgi?id=721120

win32: use the same parameter name as the implementation

Adwaita: simplify spibutton styling a bit

iconhelper: Track icon-name fallback usage in the GIcon

Passing GTK_ICON_LOOKUP_GENERIC_FALLBACK to the icon lookup doesn't work
for GIcons, so we have to make sure we use the right GThemedIcon.

Fixes image-icon-name-use-fallback reftest.

reftests: Rename reftest

After figuring out what the actual problem is, name the reftest
properly.
The actual problem is that the use-fallback property is ignored when
using an icon-name on GtkImage.

reftests: Add test for GtkImage use-fallback property

Fallback seems to be working in the GtkIconTheme test suite, but fails
in GtkImage itself.

This is a test for a bug in the Bluetooth settings. An icon named
"phone-apple-iphone" should fallback to "phone" if the
gnome-icon-theme-extras package isn't installed.

Small cleanup

Use g_clear_object in gtkdnd.c.

color chooser: Render icons sharp at scale 2

We need to load the icons at the right scale, and render them
to a surface with the right scale too.

https://bugzilla.gnome.org/show_bug.cgi?id=756195

gtkfilesystem: protect against null filesystem type

The filesystem:type attribute could be NULL, then g_strv_contains
will crash if that happens.
Just don't call it if the attribute is not set.

wayland: Allow falling back to compatibility EGL contexts

If the shared context is in legacy mode, or if the creation of a core
profile context failed, we fall back to an EGL context in compatibility
mode.

Since we're relying on a fairly new EGL implementation for Wayland, we
don't fall back to the older EGL API, and instead we always require the
EGL_KHR_create_context extension.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

docs: Improve description of gdk_gl_context_is_legacy()

Explain why this function is available, and why you may need it.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

Allow testglarea to work with legacy GL contexts

Use the 1.30 GLSL shading language for the fragment and vertex shaders.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

Control legacy GL context via environment variable

For testing purposes, we may want to force the creation of legacy GL
contexts via an environment variable.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

x11: Create legacy GLX contexts

If GLX has support for the GLX_ARB_create_context_profile extension,
then we use the GLX_CONTEXT_COMPATIBILITY_PROFILE_BIT_ARB; if it does
not, we fall back to the old glXCreateNewContext() API.

We use the shared GdkGLContext to decide whether the GLX context should
use the legacy bit or not.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

gl: Use older GLSL shaders with legacy contexts

If we're using modern GLSL, then we should stop using deprecated
modifiers, like 'varying' and 'attribute', as well as deprecated global
variables, like 'gl_FragColor'.

On the other hand, with legacy contexts we should be using older GLSL
shaders, to maximize compatibility.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

gl: Store the legacy bit in the GL program data

We need to know if we're using a legacy GL context in various places.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

gdk: Allow querying if a GL context is in legacy mode

We want to have the ability to fall back to legacy GL contexts when
creating them. In order to do so, we need to store the legacy bit on the
GdkGLContext, as well as being able to query it.

Setting the legacy bit from outside GDK is not possible; we cannot
create GL contexts in 3.2 core profile *and* compatibility modes at the
same time, and if we allowed users to select the legacy mode themselves,
it would break the creation of the GdkWindow's paint GL context.

What we do allow is falling back to legacy GL context if the platform
does not support 3.2 core profiles — for instance, on older GPUs or
inside virtualized environments.

We are also going to use the legacy bit internally, to choose which GL
API we can use when drawing GL content.

https://bugzilla.gnome.org/show_bug.cgi?id=756142

textiter: fix bug in _gtk_text_btree_get_iter_at_last_toggle()

If the last tag toggle is the end iter, the function returned the wrong
tag toggle.

This resulted in some bugs where the view wasn't relayout/redrawn
correctly.

The function also always returned TRUE, probably because the return
value is used nowhere. But for consistency with
_gtk_text_btree_get_iter_at_first_toggle(), it's better to keep the
return value, and also because otherwise the function would be wrong (it
doesn't always return a tag toggle, if there is none).

https://bugzilla.gnome.org/show_bug.cgi?id=755413

GtkPopover: Don't apply shape on wayland

Popovers are subsurfaces on wayland so we don't need that, and the shape
messed up hidpi popovers there.

wayland: Don't ignore wl_keyboard.leave if surface is gone

keyboard_handle_leave() might be called with a NULL surface resource
(for example if the surface was destroyed after the event was sent). If
so, we should still deal with the keyboard focus lost event, otherwise
we will both leak (the keyboard_focus GdkWindow reference) and miss
stopping the key repeat timer.

https://bugzilla.gnome.org/show_bug.cgi?id=755927